-
Notifications
You must be signed in to change notification settings - Fork 5.9k
chore(deps): move yarn to devDependencies #3231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😍
Codecov Report
@@ Coverage Diff @@
## main #3231 +/- ##
=======================================
Coverage 46.90% 46.90%
=======================================
Files 23 23
Lines 1196 1196
Branches 237 237
=======================================
Hits 561 561
Misses 451 451
Partials 184 184 Continue to review full report at Codecov.
|
Sweet! Feel free to merge! 🎉 |
I think this might make |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Address code-asher's comment
If it's needed for prod I guess it doesn't really make sense to remove right now... maybe a low-prio later thing. |
Fixes
yarn
being shipped in our releases removing about ~5mb from the decompressed tar.